Skip to content

docs: Add cargo fmt reminder to CLAUDE.md#2738

Merged
runningcode merged 1 commit intomasterfrom
no/add-cargo-fmt-reminder
Sep 12, 2025
Merged

docs: Add cargo fmt reminder to CLAUDE.md#2738
runningcode merged 1 commit intomasterfrom
no/add-cargo-fmt-reminder

Conversation

@runningcode
Copy link
Copy Markdown
Contributor

Summary

Add reminder to always run cargo fmt before committing Rust code changes for consistent formatting.

🤖 Generated with Claude Code

Add reminder to always run cargo fmt before committing Rust code changes.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@runningcode runningcode requested review from a team and szokeasaurusrex as code owners September 10, 2025 14:57
Copy link
Copy Markdown
Member

@szokeasaurusrex szokeasaurusrex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks okay for now, but long term, I would really like to avoid needing to maintain separate docs for each agent.

Fortunately, it looks like AGENTS.md will solve the problem. It is a new standard format for agent rules. Cursor already partially supports it, with full support (also with nested subdirectories) planned for the next minor release, 1.6. Claude Code unfortunately appears to lack support for AGENTS.md for now, though I guess they will probably add it at some point. And, in the meantime, we can import it into CLAUDE.md using the @AGENTS.md syntax or by creating a symbolic link.

I'll open an issue to migrate to AGENTS.md, but I think it makes sense to wait until Cursor supports it fully.

@runningcode
Copy link
Copy Markdown
Contributor Author

Agreed! It doesn’t make sense to maintain separate docs for each agent.

@runningcode runningcode merged commit 3728f6f into master Sep 12, 2025
23 checks passed
@runningcode runningcode deleted the no/add-cargo-fmt-reminder branch September 12, 2025 07:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants